Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DISPATCH-2144 Fatal Python error: _PyMem_DebugMalloc: Python memory allocator called without holding the GIL #1495

Merged
merged 1 commit into from
Jan 27, 2022

Conversation

jiridanek
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Jan 25, 2022

Codecov Report

Merging #1495 (c9a9771) into main (8147b62) will increase coverage by 0.07%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1495      +/-   ##
==========================================
+ Coverage   84.86%   84.94%   +0.07%     
==========================================
  Files         116      116              
  Lines       28644    28616      -28     
==========================================
- Hits        24310    24308       -2     
+ Misses       4334     4308      -26     
Impacted Files Coverage Δ
src/python_embedded.c 68.75% <100.00%> (+0.31%) ⬆️
src/server.c 87.40% <100.00%> (+0.02%) ⬆️
src/router_core/router_core.c 86.09% <0.00%> (-0.95%) ⬇️
src/router_core/delivery.c 93.71% <0.00%> (-0.74%) ⬇️
src/router_core/connections.c 90.14% <0.00%> (-0.29%) ⬇️
src/adaptors/http1/http1_server.c 85.73% <0.00%> (-0.28%) ⬇️
src/container.c 84.46% <0.00%> (-0.17%) ⬇️
src/router_node.c 94.05% <0.00%> (+0.40%) ⬆️
src/message.c 88.70% <0.00%> (+1.21%) ⬆️
src/adaptors/tcp_adaptor.c 78.90% <0.00%> (+1.47%) ⬆️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8147b62...c9a9771. Read the comment docs.

@jiridanek jiridanek marked this pull request as ready for review January 26, 2022 10:32
jiridanek added a commit to jiridanek/qpid-dispatch that referenced this pull request Jan 27, 2022
@jiridanek jiridanek force-pushed the jd_2022_01_25_DISPATCH-2144 branch from c9a9771 to dd17623 Compare January 27, 2022 09:37
@jiridanek jiridanek force-pushed the jd_2022_01_25_DISPATCH-2144 branch from dd17623 to b131cdc Compare January 27, 2022 09:38
@asfgit asfgit merged commit b131cdc into apache:main Jan 27, 2022
@jiridanek jiridanek deleted the jd_2022_01_25_DISPATCH-2144 branch January 27, 2022 09:39
ChugR pushed a commit to ChugR/qpid-dispatch that referenced this pull request Oct 13, 2022
…llocator called without holding the GIL (apache#1495) (#75)

* DISPATCH-2144 Fatal Python error: _PyMem_DebugMalloc: Python memory allocator called without holding the GIL (apache#1495)

* Remove Python lock on the router's side; use the lock Python itself created
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants